cssstylechange: Add helper function to print change

So I don't have to print both styles to the console, paste them both
into a file and then run diff on the 2 files anymore.

iconhelper: Un-duplicate some code

Remove more unnecessary redraws

The call to gtk_widget_set_state_flags immediately before these already
queues a redraw/allocate/resize in case they have to be queued.

Adwaita: No rounded corners for fullscreen

If the theme has rounded corners for fullscreen, we don't tell the
window manager that we are now fully opaque, which then makes things
less efficient than they should be.

https://bugzilla.gnome.org//show_bug.cgi?id=761571

box gadget: Fix allocation without baseline

When we don't get a baseline passed in, we want to basically
center the children inside the allocation. There was an attempt
in the code to do 'internal baseline alignment', but it had the
side effect of moving the contents to the top when we don't get
a baseline passed in. Remove it for now, this needs some more
infrastructure to do properly.

https://bugzilla.gnome.org/show_bug.cgi?id=761363

widget-factory: Update levelbar example

Set reasonable level offsets.

Updated Spanish translation

Updated Spanish translation

Fix levelbar css node tests

These were affected by the recent change to level offset handling.
At the same time, make the test files more realistic by updating
the level offsets when we set a custom range.

level bar: Document buildable syntax

Updated Polish translation

level bar: Fix offset behavior

We had some odd special-casing for the lowest and highest offset
that did not quite work. The new rule is simple: If the value
is between offset n-1 and n, it gets the style for offset n.

https://bugzilla.gnome.org/show_bug.cgi?id=761416

level bar: Improve documentation

The docs were not explaining at all what happens to existing
level offsets when the min- or max-value of a level bar are
changed.

level bar: Make the full offset official

We are adding 3 offsets, not just two. Add a define for the
third one, and mention it in the docs.

Updated Slovak translation

wayland: rework buffer management code (3 changes)

There are a couple of issues with the way that buffers are handled in
wayland in right.  These issues mean that:

 - buffers can get leaked at a fairly fast clip under the right
   conditions. This leads to the OOM killer kicking in and
   gnome-shell and gnome-terminal (for instance) showing memory
   usage in the high gigabytes range.

 - drawing can happen to a shared memory buffer at the same time
   the compositor is reading out the pixels.  This can lead to
   glitching in drawing and other undefined behavior by the compositor.

This changeset reworks how buffer management is done in the code to try
to address both problems.

The first change (commit 2c300081) addresses the leak by dropping code
that has an unchecked cairo_surface_reference call.  The code is dropped
rather than fixed, because it has a more serious issue: it's overarching
purpose is to deal with shared memory buffer contention with the
compositor, but it does it in a racy way and so fails at that mission.

The second change (commit 40e91195a) moves what layer of the code buffer
release events are handled. This is an organizational change in the
code, with no functional changes, but it's important for the last change
in the changeset.

The last change (commit c80dd549) adds back code for dealing with shared
member buffer contention in a race free way. The new code is careful to
never reuse a buffer that hasn't been explicitly released by the
compositor.

wayland: stage uncommitted changes to dedicated buffer

Right now we use one buffer for both staged changes (freshly painted
changes waiting for the frame clock to send to the compositor) and
committed changes (changes actively being read by the compositor
process). This creates a problem in the event we need to stage updates
at the same time the compositor is processing committed updates: we
can't change what the compositor is actively processing.

The current solution for handling this contention is to allocate a
temporary buffer on the spot at the time the updates are staged, and to
copy that buffer back to the shared buffer later.  The problem, though,
is that the copy to the shared buffer currently happens as soon as
the updates are finished being staged, not when the shared buffer is
done being processed by the compositor.

In order to address that problem, this commit changes the code to always
stage changes to a dedicated staging buffer.  The staging buffer is
used exclusively by the client until the client is done with it, and then
once that staging buffer is committed, the client never writes to that
buffer again.  If the client needs to stage new updates, it allocates a
brand new staging buffer, draws to it, and back fills the undrawn parts
of the buffer from a copy of the contents of the committed buffer.

As an optimization, the compositor has the option of releasing the
committed buffer back to the client.  If it does so before the client
needs to stage new updates, then the client will reuse the buffer
for staging future updates.  This optimization prevents having to allocate
a new staging buffer and the associated cost of back filling
that new buffer with a readback of the committed buffer.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: don't handle buffer release centrally

Right now we handle buffer releases coming from the
compositor in a central place. We add a listener when
first creating the shared buffers.

This is problematic because a buffer can only have
one listener on it at once so users of the buffer
can't get notified when it's released.

This commit moves the buffer listener code from the
centrally managed display code to the cursor and window
code.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: always return FALSE from begin_paint

The client and compositor share access to the window
pixel buffers. After the client hands off (commits)
the buffer to the compositor it's not supposed to write
to it again until it's released by the compositor.

The code tries to deal with this contention by allocating
a temporary buffer and using that in the mean time. This
temporary buffer is allocated by a higher layer of the code
when begin_paint returns TRUE. Unfortunately, that layer of
the code has no idea when the buffer is released, so it ends
up blitting the temporary buffer back to the shared buffer
prematurely.

This commit changes begin_paint to always return FALSE.

A future commit will address the contention problem in
a different way.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: use g_clear_pointer when destroying cairo surfaces

There are a few places where we destroy a cairo surface and
then nullify it. This commit changes those to use
g_clear_pointer instead.

It also drops a cairo_surface_finish call that is unnecessary

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: rename cairo surface user data key to be more specific

This commit renames the key name to be more specific for clarity.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: move server proxy objects to substructure

This commit moves the server proxy objects to a substructure
for clarity.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: rename window->surface to window->wl_surface

The name surface is really overloaded when dealing
with wayland windows.

To alleviate ambiguity, this commit changes the name
of the "surface" and "subsurface" members to have
a wl_ prefix.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

Fix stylecontext tests

This was broken by f7ec9c98ef0ef8740c93f96af9d971b0211118c1,
since type names are no longer used at all in CSS matching.

Update CSS docs regarding type names

We no longer use type names at all.

cssselector: Remove type references

We don't use types anymore.

cssmatcher: Remove type argument from name matcher

We no longer use types when matching names.

inspector: Drop type name from node list

It's not used anymore in the CSS, so people don't need to see it.

cssimageradial: Only allow at "<position>" after other props

Don't allow syntax like
  at top left circle
but follow the spec about requiring the at <position> right before the
comma.

This is porbably because
  circle at 10px 10px
could be interpreted as
  circle 10px at 10px
with the now disallowed syntax, too.

Test included.

cssimageradial: Only emit one error for wrong position

Test included.

css tests: Fix style tests

Commit d0fd7990c3f2c738f86185418ccd76b14da8a776 changed
the location of builtin themes, breaking the loading of
the empty theme in test-css-style.

css tests: Fix expected output

This is fallout from f7ec9c98ef0ef8740c93f96af9d971b0211118c1.

css: Convert numbers with GTK's dpi

Don't hardcode 96 for dpi, but instead use the value of the -gtk-dpi
property (that mirrors the GdkScreen's dpi if it wasn't set explicitly).

This makes these values scale when the large font setting in
control-center is enabled.

css: Properly convert em, ex and rem to pixels

The previous code converted to pt.

Adwaita: darker background for popovers

levelbar: Make discrete levelbars behave as before

During the gadget conversion, the drawing of discrete levelbars
was unintentionally changed to draw a wide trough but narrow
blocks, which does not look great. So go back to the previous
way of drawing things.

https://bugzilla.gnome.org/show_bug.cgi?id=761428

Fix indentation

gcc 6 complains about this.

Document rem as CSS unit

switch: Don't queue draws after setting state flags

css: Add rem unit

Good luck, Lapo.

widget: Set a css name

Now selecting a widget by class name no longer works.

This is probably most relevant for users outside of GTK that want to
style their own widgets. Those widgets should now either add their own
style classes (if they want to adjust existing CSS) or use
gtk_widget_class_set_css_name() themselves (if they want to get rid of
all "upstream" styling).

Range: Group all three gestures

And reset the grab_location in the ::released handler of the multipress
gesture.

Previously, when leaving fine-tune mode, the ::released handler of the
multipress gesture would call stop_scrolling, which calls
range_grab_remove and resets the grab_location. The ::drag-end handler
is executed after that, and only unsets priv->in_drag if the
grab_location is MOUSE_OUTSIDE, which it never was, since the ::released
handler already reset it. This lead to priv->in_drag being set even
though no dragging was in progress anymore, which e.g. made shift
pressed after leaving the fine-tune mode entering it again.

https://bugzilla.gnome.org/show_bug.cgi?id=761402

x11: don't apply scaling factor twice

The fallback behaviour of get_work_area () divides the
screen width and height by the window scaling factor, but
those values are already scaled down.

https://bugzilla.gnome.org/show_bug.cgi?id=761474

Don't do remote checks on NULL files

Calling _gtk_file_consider_as_remote() with a NULL argument
results in warnings being thrown.

Note that query->priv->location being NULL is a state that does
not seem to be invalid by itself.

This could happen if you do search-as-you-type in a filechooser,
which has a filter that does not match anything *and* the current
"place" selected is "Recent".

https://bugzilla.gnome.org/show_bug.cgi?id=761552

Move private widget path functions away

We don't put private functions in public headers anymore.

https://bugzilla.gnome.org/show_bug.cgi?id=761339

win32: Get rid of XP theme

XP is not supported anymore. And we can massively simplify the win32
theme if we don't have to support 2 versions.

win32: Delete classic theme

Well, actually, stop adding an extra version of raleigh

build: Don't break windows build

Hardcode the existing files instead of doing a for loop and attempting
string-fudging magic.

Adwaita: removed some old cruft from _drawing.scss

resulting in a nice number of non working selectors ditched.

inspector: Don't hardcode builtin themes

Insteead, enumerate the resources that are available. Fixes windows
themes not being listed in the inspector.

cssprovider: Change location of builtin themes

Instead of
  /org/gtk/libgtk/theme/$THEME-$VARIANT.css
look at
  /org/gtk/libgtk/theme/$THEME/gtk-$VARIANT.css
and that way mirror the directory layout of real themes.

actionbar: Remove unnecessary map/unmap implementations

These just do what the default implementations do anyway.

statusbar: Remove unused include

Remove _gtk_box_get_children

It's doing exactly the same thing as gtk_container_get_children.

colorswatch: Remove unused constant

entry: queue an allocate when pulsing

We need to reposition the progress gadget and that's done during
allocation.

gdkwindow-x11: always set _GTK_THEME_VARIANT property

Set _GTK_THEME_VARIANT to empty string when default theme variant
is used. This will allow to understand whether _GTK_THEME_VARIANT
is not supported or default variant is requested.

https://bugzilla.gnome.org/show_bug.cgi?id=761476

gtksettings: fix critical warning

Commit 5186aeb90f52c941a2642b04ebfb54e9d8f8ea11 introduced critical
warning - g_value_copy: assertion 'g_value_type_compatible (...)' failed.

https://bugzilla.gnome.org/show_bug.cgi?id=761381

broadway: Define variables where they are used

Avoids gcc warnings about unused variables.

composetable: Fix gcc warnings

immodule: Use correct sign for function prototype

entry: Turn progress node into a gadget

entry: Simplify drawing code

Adwaita: fix headerbar spinbutton size

fixes https://bugzilla.gnome.org/show_bug.cgi?id=761484

gdkborder: remove unneeded definition

This reverts 54fa65f544bef281640d18a4a725005f6199a956,
5190d010d95010a845715c719c3b85a75e0c513a, and
a6e4de28842002e7bfdf73def26c4c52d9f9126a.

https://bugzilla.gnome.org/show_bug.cgi?id=756579

print dialog: Fix layout preview drawing

Use the CSS name paper for this as well, and make sure Adwaita
sets a color that makes the text visible in the dark theme as well.

Adwaita: notebook with hidden header style fix

the stack styling is applied only when there is a header, hence
when the stack node is not the notebook only child, should cure
https://bugzilla.gnome.org/show_bug.cgi?id=760996

Adwaita: fix vertical spinbutton style

correct sizing and workaround disabled entry text colors.

notebook: Do not reorder tabs after a grab notify if there isn't an ongoing reorder operation

This broke in f535251507a6a0af68aed6348ae80cbdac9b1f4a that removed
during_reorder in favor of using the current operation, but removed the
check from gtk_notebook_stop_reorder().

Updated French translation
(cherry picked from commit 85fb4c1c0a17906a65e0b732770b88353b6214c4)

gtksettings: reset property to correct default value

https://bugzilla.gnome.org/show_bug.cgi?id=76138

Revert "Add missing (out) annotation to gdk_rgba_parse"

This reverts commit 695860958a9a2e7c825bcec4d0d42b427f0fb47e.

The change introduced an introspection ABI break.

See also: https://bugzilla.gnome.org/show_bug.cgi?id=682125

If we want to move the parse() method to be a constructor or a class
method, we'll need to do more testing or add new API.

https://bugzilla.gnome.org/show_bug.cgi?id=754990

entry: Draw progress in the right place

Updated Hungarian translation

Updated German translation

reftests: Make separator-size test pass in HighContrastInverse

After neutralizing the stack background, the notebook itself
was peeking through.

Updated Spanish translation

Updated Spanish translation

Trivial formatting fix

reftests: Make label-text-shadow-changes-modify-clip test more reliable

This test was failing in continuous, where the tests are running
in a VM with disabled animations. Make the test adapt to that
situation by changing the rui on the fly if we find that animations
are disabled.

reftests: Add a message to the test output

Emit a message that ends up in the test output if we are modifying
the test on the fly.

reftests: Rename a source file

Easier to keep track of things if the sources are names after
the test they are used in.

entry: Fix icon areas

Do the same thing we already do in gtk_entry_get_text_area.

widget: Annotate out params of _translate_coordinates as optional

reftests: Make animation-direction test more reliable

This test was failing in continuous, where the tests are running
in a VM with disabled animations. Make the test adapt to that
situation by changing the reference ui on the fly if we find
that animations are disabled.

gtk-builder-tool: Expand the preview command

Make the preview command parse options properly, turn the ID into
an --id=ID option, and add a --css=FILE option that allows to
specify a css file to use for previewing.

This is useful for e.g. previewing the reftest .ui files with
their corresponding .css.

gtk-launch: Add --version

Support --version. Also improve the --help output cosmetically.

gtk-builder-tool: Accept --help

Saying 'Failed to open file '--help' is not polite or smart.
Catch this case and show usage information.

file chooser: Avoid a crash

Since 39c2d12330b6d4405ca8a5599c12017c58626fcf,
priv->operation_mode == OPERATION_MODE_BROWSE no longer
guarantees that priv->browse_files_model is the current
model of the list - we are only switching the models after
loading the new directory. Avoid triggering the assertion
in show_and_select_files by checking if we have right model
before calling it.

https://bugzilla.gnome.org/show_bug.cgi?id=761209

Don't include the builtin sequence table in multiple places

The table is defined as static const guint16 [], so it really
should only be included in one source file.

combobox: Avoid a misleading indentation

gcc 6 warns about this sort of thing. There were also some
stray ; here.

Fix the border-image-excess-size reftest

This was failing because the grid is now respecting border
width too. Making the selector more specific so it only
applies to the toolbars fixes this.

Fix another reftest

Make sure the notebook in the separator-size reftest does not
peek through, by giving it a transparent background.

Fix some reftests under wayland

Tests need to use popup windows, in order to avoid differences
due to CSD.

wayland: Reduce warning level for parentless temporaries

Temp windows without parent are used frequently in our testsuite;
using g_warning on them causes the tests to fail, which is not
useful. Reduce the warning to a g_message.

treeview: Draw background when empty

We have a function we call for drawing the empty case, but
it only renders a focus rectangle. Make it draw the background
as well.

https://bugzilla.gnome.org/show_bug.cgi?id=761309

colorchooserwidgeet: Reset the selected swatch when removing

Otherwise, priv->current points to garbage and subsequently leads to a
crash, e.g. when adding another custom color.

Document gtk_parse_args limitations

Mention that some things won't work without a display
connection.

Throw an error when a display is missing

Creating style context won't work without a display, so
give a clear error message and abort instead of segfaulting
later on.

https://bugzilla.gnome.org/show_bug.cgi?id=761332

3.19.8